- 
                Notifications
    You must be signed in to change notification settings 
- Fork 6.2k
Fix JdbcUserCredentialRepository Save #16621
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, @franticticktick, I've left some feedback inline.
        
          
                .../java/org/springframework/security/web/webauthn/management/JdbcUserCredentialRepository.java
              
                Outdated
          
            Show resolved
            Hide resolved
        
      | Also, will you please rebase to  | 
| Hi @jzheaux . Thanks for your feedback. This component was introduced in current version, so it is not in  | 
2f41603    to
    af56c22      
    Compare
  
    There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @franticticktick! I've left a question for you inline.
        
          
                .../java/org/springframework/security/web/webauthn/management/JdbcUserCredentialRepository.java
              
                Outdated
          
            Show resolved
            Hide resolved
        
      | Thanks, @franticticktick! Can you squash your commits please? Then I think we're ready to merge. | 
Closes spring-projectsgh-16620 Signed-off-by: Max Batischev <[email protected]>
293883e    to
    e28b9bd      
    Compare
  
    | @jzheaux sory please, i usually don't miss things like that. | 
| Thanks, @franticticktick! This is now merged into  | 
Closes gh-16620